Skip to content

Assignee 설정 자동화 #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 29, 2024
Merged

Assignee 설정 자동화 #30

merged 1 commit into from
Apr 29, 2024

Conversation

DaleSeo
Copy link
Member

@DaleSeo DaleSeo commented Apr 28, 2024

#26 (comment) 에서 논의되었던 것처럼, 현재 권한 문제로 코드 작성자가 스스로 자신을 Pull Request에 Assignee로 설정할 수가 없습니다. 기여자 경험을 개선하고 관리자가 일일이 Assignee를 설정하는 수고가 없도록, GitHub Actions에 자동화 워크플로우를 추가하려고 합니다. 이 PR이 main 브랜치에 머지되면, Pull Reqeust에 Assignee가 자동으로 설정이 될 것입니다.

@DaleSeo DaleSeo self-assigned this Apr 28, 2024
@DaleSeo DaleSeo changed the title ci: assign issues and pull requests automatically Assignee 설정 자동화 Apr 28, 2024
@DaleSeo DaleSeo force-pushed the auto-author-assign branch from 3ba7a24 to 870adf3 Compare April 28, 2024 18:41
Copy link
Member

@WhiteHyun WhiteHyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다 :)

@WhiteHyun
Copy link
Member

저는 Approve 권한이 없군요. 이런 😂

@DaleSeo
Copy link
Member Author

DaleSeo commented Apr 28, 2024

저는 Approve 권한이 없군요. 이런 😂

엄밀히 얘기하면, 누구나 PR을 승인할 수 있습니다. 단지, Merge pull request 버튼이 활성화가 안 되는 것 뿐이죠. 오픈 소스 협업 모델을 따르고 있기 때문에, 저를 포함한 저장소 maintainer가 추가로 승인을 해야 PR을 머지할 수 있게 설정이 되어 있습니다. 앞으로 다른 분들의 코드 리뷰를 잘 해주시는 스터디 멤버 분들께는 저장소에 write 할 수 있는 권한을 드릴테니 열심히 활동해주세요 😄

@leokim0922
Copy link
Contributor

좋은 방법인거 같습니다! 승인하도록 하겠습니다!

@DaleSeo DaleSeo merged commit 28d3c92 into main Apr 29, 2024
@DaleSeo DaleSeo deleted the auto-author-assign branch April 29, 2024 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants